Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the same request independently of the order that the models are loaded #1019

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aronwolf90
Copy link

@aronwolf90 aronwolf90 commented Feb 3, 2022

Currently, the order of the index in a search request changes
with the order that the models are loaded.

Problem: This does not matter for real rest request but
makes problems with mocks (E.g. VCR gem) on tests. The order of the tests can change
and in consequence the autoloader from rails loads models in a other order, causing that the
mock does not longer match since the order of indexes in the rest request not longer matches.

…loaded

Currently, the order of the index in  a search request changes
with the order that the models are loaded.

Problem: This does not matter for real request but
makes problems with mocks (E.g. VCR gem) on tests when the
order of the tests changes.
@aronwolf90 aronwolf90 closed this Feb 3, 2022
@aronwolf90 aronwolf90 reopened this Feb 3, 2022
@aronwolf90 aronwolf90 changed the title Make the same request independently of the order that the models are … Make the same request independently of the order that the models are loaded Feb 3, 2022
@aronwolf90
Copy link
Author

Do I have forgotten something to do? Is something unclear?

@aronwolf90
Copy link
Author

?

@aronwolf90
Copy link
Author

Ping

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant